-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add smart proxies #146
Add smart proxies #146
Conversation
Or should we add the proxies on Host? |
Extract PuppetProxy and PuppetCAProxy. Extracted in: theforeman/foreman_puppet#146
155215d
to
0518651
Compare
I'm not sure if I'm familiar enough with the code, but can we delegate the methods from |
We definitelly can, but I'm not sure if that's the best way forward as there is specialized API for plugins to add their proxies to |
Yep, I'm quite familiar with the API as I initially wrote it. :-D I'm all for using that API to register the proxies, that makes a lot of sense imho. |
Extract PuppetProxy and PuppetCAProxy. Extracted in: theforeman/foreman_puppet#146
Looks like this is still part of core. Do we need this PR? |
Extract PuppetProxy and PuppetCAProxy. Extracted in: theforeman/foreman_puppet#146
0518651
to
b12cbec
Compare
b12cbec
to
789713a
Compare
c0d88ea
to
581f4aa
Compare
I believe this should get in, but only for Foreman 3.1 release. |
@ezr-ondrej: Do you mind creating a milestone for this so we have better visibility? Thanks. |
Closing the PR since there haven't been any updates for almost two years. |
No description provided.